-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix automerge action #10055
Fix automerge action #10055
Conversation
Quoting the README of the action
I overwrite the number of required reviews to 0, because "we" automatically approve the merge if all tests are green.
|
Current output:
If I understand correctly, that means, if if |
Adding label "dependencies" (and closing and reopening) to trigger automege |
Note that GitHub's automerge functionality is no option for us. Sometimes, we want to have manual merges. Only in the case of version updates, we want to have automatic ones. |
Blocked by pascalgn/automerge-action#225 (comment) I merge nevertheless to fix the label |
We want to keep the rule of two reviewers requried. Thus, we cannot apply pascalgn/automerge-action#200 (comment). |
Co-authored-by: Houssem Nasri <[email protected]>
Last commit tries to make use of pascalgn/automerge-action#61. |
The build for this PR is no longer available. Please visit https://builds.jabref.org/main/ for the latest build. |
Closing this because it's not that important and blocked by the other repo |
Currently, all auto merge werks. Using GitHub's CLI tool. See https://github.com/JabRef/jabref/blob/main/.github/workflows/automerge.yml. |
We are having bots creating Pull Requests. Example: #10054. We have an action for auto-merging these PRs if they are OK. The bot, however, does not work currently. This PR tries to fix it.
Mandatory checks
CHANGELOG.md
described in a way that is understandable for the average user (if applicable)